Skip to content

fix: contain panics across persisted loads - #49

Merged
pathscale merged 1 commit into
masterfrom
fix/torn-space-parse-containment
Aug 4, 2026
Merged

fix: contain panics across persisted loads#49
pathscale merged 1 commit into
masterfrom
fix/torn-space-parse-containment

Conversation

@pathscale

@pathscale pathscale commented Aug 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • contain panics from lower-level persisted-page decoding and WorkTable reconstruction
  • convert both dependency panics and ordinary decode errors into PersistenceLoadError
  • cover the containment boundary with deterministic unit tests
  • keep both crates at 1.0.0-beta.4

This addresses the all-features master failure in run 30957623797, where data_bucket panicked while parsing a torn unsized index page.

Validation

  • cargo fmt --all
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • exact regression: persistence::torn_shutdown::test_store_survives_torn_shutdowns
  • cargo test --workspace --all-targets --all-features (all passed; 4 documented ignores)

Closes #41 once merged and green.

@pathscale
pathscale merged commit 72b6e81 into master Aug 4, 2026
5 checks passed
@pathscale
pathscale deleted the fix/torn-space-parse-containment branch August 4, 2026 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Define and test the 1.0 persistence durability and crash-recovery contract

1 participant